feat: Add startup-time config validator with deprecation mapping#881
Open
SmartDever02 wants to merge 1 commit intobase:mainfrom
Open
feat: Add startup-time config validator with deprecation mapping#881SmartDever02 wants to merge 1 commit intobase:mainfrom
SmartDever02 wants to merge 1 commit intobase:mainfrom
Conversation
Implements issue base#783: Add config validator to detect configuration issues before containers start. Features: - Validates required environment variables (common + client-specific) - Detects deprecated/renamed environment variables with replacement suggestions - Identifies unknown/typo variables with similarity matching - Validates URL, port, and numeric formats - Provides consolidated error/warning report in single output Usage: - make doctor [ENV_FILE=.env.mainnet] - ./scripts/validate-config.sh [env-file] - docker compose --profile validation run --rm config-doctor This solves the problem of: - Scattered errors across container logs - No clear guidance on fixing configuration issues - Time-consuming diagnosis of misconfigurations - Silent failures that only appear later The validator can be run before starting services to catch issues immediately with actionable suggestions.
Collaborator
🟡 Heimdall Review Status
|
Author
|
could you review my pr plz? @mur-me |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements issue #783: Add config validator to detect configuration issues before containers start.
Features:
Usage:
This solves the problem of:
The validator can be run before starting services to catch issues immediately with actionable suggestions.
Contribution by Gittensor, see my contribution statistics at https://gittensor.io/miners/details?githubId=102175066